Skip to content

Ignore statements that have no query id - #840

Merged
artemgavrilov merged 1 commit into
mainfrom
no_query_id
Sep 3, 2026
Merged

Ignore statements that have no query id#840
artemgavrilov merged 1 commit into
mainfrom
no_query_id

Conversation

@artemgavrilov

@artemgavrilov artemgavrilov commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Match pg_stat_statements behavior and ignore statements without query id. Otherwise all such statements stats will be merged into single entry as query id part of the hash key.

PG-0

Description

Links

@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 88.50%. Comparing base (290192b) to head (1ea0e03).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #840      +/-   ##
==========================================
+ Coverage   88.42%   88.50%   +0.08%     
==========================================
  Files           3        3              
  Lines        1339     1340       +1     
  Branches      194      196       +2     
==========================================
+ Hits         1184     1186       +2     
  Misses         75       75              
+ Partials       80       79       -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Match pg_stat_statements behavior and ignore statements without query
id. Otherwise all such statements stats will be merged into single
entry as query id part of the hash key.
@artemgavrilov
artemgavrilov marked this pull request as ready for review September 2, 2026 20:41
Comment thread regression/expected/level_tracking_4.out

@jeltz jeltz left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want this in 2.4.0?

@artemgavrilov

Copy link
Copy Markdown
Contributor Author

Do you want this in 2.4.0?

I think we can keep it for the next release. It's not good, but not critical.

@artemgavrilov
artemgavrilov merged commit cd866ff into main Sep 3, 2026
66 of 67 checks passed
@artemgavrilov
artemgavrilov deleted the no_query_id branch September 3, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants